People: drop role section headers; one unified card flow#47
Merged
Conversation
Cards now appear in YAML order (PI first, then each role group in the order defined in people.yml). The per-role <h2> headers (Principal Investigator / Graduate Students / Postdocs / Research Associates / Undergraduate Students) are gone — visitors see the individual person.title on each card instead. Former members section is unchanged (still grouped by role with h3 subheadings since alumni are listed as text, not cards). Note: John's entry in _data/people.yml has no `title` field, so his card no longer has any role context shown. Add `title: Principal Investigator` to his entry if you want it labeled.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Drop the role-based section headers on the people page ("Principal Investigator:", "Postdocs:", "Graduate Students:", "Research Associates:", "Undergraduate Students:"). Current members now flow as one unified grid in YAML order:
people.pi)Each card still displays the individual
titlefield under the name, so role context is preserved at the per-person level.Former members section is unchanged (still grouped by role with
<h3>subheadings — those are listed as text, not cards).Test plan
bundle exec jekyll buildclean_site/people.htmlNote worth addressing
John's card no longer shows any role context because his entry in
_data/people.ymlhas notitlefield — the section header was carrying that information before. Addingtitle: Principal Investigatorto his entry would label him; one-line YAML edit, not in this PR since it's an editorial decision (you might want a different label or none at all).https://claude.ai/code/session_01S5QXfkxZBNSAf2Y1XAD8H7
Generated by Claude Code